Hello! I am a robot who works on Magic Modules PRs.

I have detected that you are a community contributor, so your PR will be assigned to someone with a commit-bit on this repo for initial review.

Thanks for your contribution! A human will be with you soon.

@rileykarson, please review this PR or find an appropriate assignee.

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 5 files changed, 18 insertions(+), 12 deletions(-))
Terraform Beta: Diff ( 7 files changed, 20 insertions(+), 13 deletions(-))

I looked at all the schema entries manually loaded in MM and I have a few questions:

• source_disk_link in snapshot.erb(google_compute_snapshot). This is not documented in the provider and all it seems to be doing is converting selflinks to a one with v1 in the url.
• KMS self_link also needs deprecating as it is identical to id and KMS API doesn't have self_links.

I'll deprecate both and open issues to remove them

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 8 files changed, 25 insertions(+), 14 deletions(-))
Terraform Beta: Diff ( 9 files changed, 26 insertions(+), 14 deletions(-))

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 10 files changed, 27 insertions(+), 16 deletions(-))
Terraform Beta: Diff ( 12 files changed, 29 insertions(+), 17 deletions(-))

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 10 files changed, 27 insertions(+), 16 deletions(-))
Terraform Beta: Diff ( 11 files changed, 28 insertions(+), 16 deletions(-))

bump